Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[18.0][MIG] hr_expense_cancel #281

Open
wants to merge 27 commits into
base: 18.0
Choose a base branch
from

Conversation

mav-adhoc
Copy link

No description provided.

@mav-adhoc mav-adhoc force-pushed the 18.0-mig-hr_expense_cancel branch 4 times, most recently from eb7cb1c to 0b8f9c0 Compare December 27, 2024 18:50
@mav-adhoc mav-adhoc force-pushed the 18.0-mig-hr_expense_cancel branch 8 times, most recently from 63f2a04 to 1650425 Compare January 9, 2025 12:28
@mav-adhoc mav-adhoc force-pushed the 18.0-mig-hr_expense_cancel branch from 1650425 to 7fb11e9 Compare January 9, 2025 14:53
ernestotejeda and others added 17 commits January 29, 2025 12:18
This module lets to cancel and correct expenses. It adds a cancel
button on the expense sheet that undo reconciliations and delete
payments and journal entries.
The multi own account test has been removed as it is no longer possible
in odoo core to register the payment of several expense sheet reports at
the same time.

Co-Authored By:
Stefan Ungureanu
aiendry-aktivsoftware
In this version, if a product doesn't have cost, you are only able
to enter total amount, so the test should be adjusted to this
condition.
oca-ci and others added 6 commits January 29, 2025 12:18
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: hr-expense-17.0/hr-expense-17.0-hr_expense_cancel
Translate-URL: https://translation.odoo-community.org/projects/hr-expense-17-0/hr-expense-17-0-hr_expense_cancel/
Currently translated at 100.0% (4 of 4 strings)

Translation: hr-expense-17.0/hr-expense-17.0-hr_expense_cancel
Translate-URL: https://translation.odoo-community.org/projects/hr-expense-17-0/hr-expense-17-0-hr_expense_cancel/it/
@mav-adhoc mav-adhoc force-pushed the 18.0-mig-hr_expense_cancel branch 2 times, most recently from f3e2b0d to 734c83a Compare January 29, 2025 15:45
@mav-adhoc mav-adhoc force-pushed the 18.0-mig-hr_expense_cancel branch from 734c83a to 7f238e3 Compare January 29, 2025 15:45
@mav-adhoc mav-adhoc force-pushed the 18.0-mig-hr_expense_cancel branch from 11066e9 to 369eab5 Compare January 29, 2025 16:46
@mav-adhoc mav-adhoc force-pushed the 18.0-mig-hr_expense_cancel branch 2 times, most recently from 2a46bc1 to 029b582 Compare January 31, 2025 17:13
@mav-adhoc mav-adhoc force-pushed the 18.0-mig-hr_expense_cancel branch from 029b582 to aa5d2fb Compare January 31, 2025 18:20
Copy link

@baf-adhoc baf-adhoc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functional review ok

Copy link

@ced-adhoc ced-adhoc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@ced-adhoc
Copy link

Hi @Saran440 , could you please check this pr? thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.